-
Notifications
You must be signed in to change notification settings - Fork 99
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Optional fields default to none #194
base: main
Are you sure you want to change the base?
Conversation
Thanks to the prior work of @adriangb: 1Password#25
To the repo maintainers: this can be closed, this is only an improvement that will be included as part of #169 |
@amandayu1 It does look like #169 will include these changes. Do you object to me closing this PR as @hculea suggested? |
@savannidgerinel - for more context around this - this is not an open-source contribution, as Amanda was an intern with the Programmatic Interfaces team in the summer term. This was part of the team's sprint, and this work was eventually included in the other PR that I referenced. Also, since her internship ended, I believe Amanda no longer has access on the fork that contains this PR's source branch (which is under my namespace), so I think this should be safe to close 😄 |
No description provided.